Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Keyring migrate command doc #8979

Merged
merged 17 commits into from
Apr 12, 2021
Merged

Conversation

cyberbono3
Copy link
Contributor

Description

This is draft PR that describes keyring migrate command.

closes: #8769


Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the module structure standards.
  • Wrote unit and integration tests
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer
  • Review Codecov Report in the comment section below once CI passes

@cyberbono3 cyberbono3 requested a review from amaury1093 March 24, 2021 04:00
docs/migrations/keyring.md Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
Copy link
Contributor

@amaury1093 amaury1093 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm @cyberbono3, thanks! is this r4r?

docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
@amaury1093 amaury1093 added backport/0.42.x (Stargate) T:Docs Changes and features related to documentation. labels Mar 29, 2021
@cyberbono3 cyberbono3 force-pushed the cyberbono3/8769-add-migration-keyring-doc branch 3 times, most recently from e691eea to 61ea86f Compare March 29, 2021 21:24
@cyberbono3 cyberbono3 force-pushed the cyberbono3/8769-add-migration-keyring-doc branch from 61ea86f to 56052b7 Compare March 29, 2021 21:29
@cyberbono3
Copy link
Contributor Author

@AmauryM Can you review it, please?

@amaury1093
Copy link
Contributor

@cyberbono3 In the SDK, we generally don't put the ready-for-review label, but just put the PR as "open", which means that other people are invited to review it.

@amaury1093 amaury1093 marked this pull request as ready for review March 30, 2021 09:28
@cyberbono3
Copy link
Contributor Author

@cyberbono3 In the SDK, we generally don't put the ready-for-review label, but just put the PR as "open", which means that other people are invited to review it.

OK. Thanks for your clarification.

@cyberbono3
Copy link
Contributor Author

@AmauryM Do you know how to fix rosetta?

@amaury1093
Copy link
Contributor

@AmauryM Do you know how to fix rosetta?

Not really... #8772

@amaury1093 amaury1093 requested a review from clevinson March 31, 2021 09:30
Copy link
Contributor

@barriebyron barriebyron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if we should write a numbered list of how to migrate keypairs from Launchpad to Stargate?

  1. simd keys migrate <old_home_dir> --dry-run true
  2. (how do we test that the keypairs migrated?)
  3. simd keys migrate <old_home_dir> --dry-run false

docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
docs/migrations/keyring.md Outdated Show resolved Hide resolved
cyberbono3 and others added 8 commits April 6, 2021 12:32
Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>
Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>
Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>
Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>
Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>
Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>
Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>
@cyberbono3 cyberbono3 requested a review from barriebyron April 6, 2021 19:47
docs/migrations/keyring.md Outdated Show resolved Hide resolved
cyberbono3 and others added 2 commits April 7, 2021 11:21
@amaury1093 amaury1093 added the A:automerge Automatically merge PR once all prerequisites pass. label Apr 12, 2021
@mergify mergify bot merged commit d5f5fe6 into master Apr 12, 2021
@mergify mergify bot deleted the cyberbono3/8769-add-migration-keyring-doc branch April 12, 2021 10:18
@amaury1093
Copy link
Contributor

@Mergifyio backport release/v0.42.x

mergify bot pushed a commit that referenced this pull request May 3, 2021
* draft

* update

* address comments, ready for review

* delete thr last line

* Update docs/migrations/keyring.md

Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>

* Update docs/migrations/keyring.md

Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>

* Update docs/migrations/keyring.md

Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>

* Update docs/migrations/keyring.md

Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>

* Update docs/migrations/keyring.md

Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>

* Update docs/migrations/keyring.md

Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>

* Update docs/migrations/keyring.md

Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>

* ready for review

* Update docs/migrations/keyring.md

Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com>

Co-authored-by: SaReN <sahithnarahari@gmail.com>
Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>
Co-authored-by: Alessio Treglia <alessio@tendermint.com>
Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com>
(cherry picked from commit d5f5fe6)
@mergify
Copy link
Contributor

mergify bot commented May 3, 2021

Command backport release/v0.42.x: success

Backports have been created

alessio pushed a commit that referenced this pull request May 3, 2021
Co-authored-by: SaReN <sahithnarahari@gmail.com>
Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>
Co-authored-by: Alessio Treglia <alessio@tendermint.com>
Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com>
(cherry picked from commit d5f5fe6)
Co-authored-by: Andrei Ivasko <cyberbono3@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A:automerge Automatically merge PR once all prerequisites pass. T:Docs Changes and features related to documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update keyring migrate documentation for v0.42 and version specific migrations
4 participants